Skip to content

feat: fix dataZoom range factor #4523#4524

Closed
da730 wants to merge 1 commit intodevelopfrom
feat-brush-zoom
Closed

feat: fix dataZoom range factor #4523#4524
da730 wants to merge 1 commit intodevelopfrom
feat-brush-zoom

Conversation

@da730
Copy link
Copy Markdown
Contributor

@da730 da730 commented Mar 23, 2026

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Refactoring
  • Update dependency
  • Code style optimization
  • Test Case
  • Branch merge
  • Release
  • Site / documentation update
  • Demo update
  • Workflow
  • Other (about what?)

🔗 Related issue link

close #4523

🔗 Related PR link

🐞 Bugserver case id

💡 Background and solution

brush组件在开启dataZoom & zoomAfterBrush,每次brush后计算dataZoom的range没有考虑axis.scaleRangeFactor,导致zoom after brush的范围不对。参考未开启data zoom组件的zoom after brush计算逻辑进行fix,提取了公共逻辑计算dataZoom组件的start and end percent

📝 Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese
fix #4523

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@da730 da730 requested review from skie1997 and xuefei1313 and removed request for skie1997 March 23, 2026 07:10
@skie1997 skie1997 closed this Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] 开启zoomAfterBrush后连续brush,datazoom range仍然按照初始范围计算

2 participants